Skip to content

fix: move university card border style to inner div#391

Merged
manNomi merged 2 commits intomainfrom
fix/css-alls
Jan 25, 2026
Merged

fix: move university card border style to inner div#391
manNomi merged 2 commits intomainfrom
fix/css-alls

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Jan 25, 2026

관련 이슈

  • resolves: #이슈 번호

작업 내용

  • 메인페이지 대학 카드 border 깨짐 버그 수정

특이 사항

리뷰 요구사항 (선택)

@vercel
Copy link

vercel bot commented Jan 25, 2026 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
solid-connection-web Ready Ready Preview, Comment Jan 25, 2026 4:39pm

@coderabbitai
Copy link

coderabbitai bot commented Jan 25, 2026 •
edited
Loading

Warning

Rate limit exceeded

@manNomi has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 20 minutes and 11 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

Walkthrough

UniverSityCard 컴포넌트의 스타일 구조를 재정리하는 작업입니다. 기존에는 Link 요소가 모든 스타일링을 담당했으나, 이제는 두 겹의 래퍼로 나누어 책임을 분담합니다. 구체적으로는:

  1. Link 요소의 className을 단순한 "block"으로 축약
  2. 외부 래퍼 div에 테두리와 호버 효과 이관
  3. 내부 래퍼 div에 레이아웃 관련 클래스 배치

사용자 상호작용 관점에서는 동작이 동일하게 유지되지만, 스타일링 적용 범위가 재구성되었습니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • wibaek
  • khwww
🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive 필수 섹션 구조는 있지만, '관련 이슈' 항목이 불완전하고 '특이 사항' 및 '리뷰 요구사항' 섹션이 비어 있어 설명이 부분적으로 미흡합니다. 구체적인 이슈 번호를 명시하고, 특이 사항이나 리뷰 중점 사항이 있다면 작성해 주세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경사항의 핵심을 명확하게 반영하고 있습니다. 대학 카드의 border 스타일을 내부 div로 이동하는 구체적인 변경 내용을 정확히 설명합니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@manNomi manNomi merged commit 94ccca7 into main Jan 25, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wibaek wibaek Awaiting requested review from wibaek wibaek is a code owner

@enunsnv enunsnv Awaiting requested review from enunsnv enunsnv is a code owner

@khwww khwww Awaiting requested review from khwww khwww is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@manNomi